Multiple Dependency Libraries #259
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for the fix! FWIW, I'm in the process of migrating all of the robotpy libraries to a bunch of hatchling plugins and eventually will archive this repo:
robotpy/mostrobotpy#170 is a PR where I migrated the robotpy core libraries from robotpy-build to semiwrap. |
|
Circling back on this, I've started migrating the various vendor libraries over to semiwrap, so they're a lot simpler to understand the differences between robotpy-build and semiwrap et al. See robotpy/robotpy-playingwithfusion#19 for an example. If you have other questions, fixes, etc, please reach out! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If multiple downloads are specified in the pyproject.yaml (e.g. [[tool.robotpy-build.wrappers."".download]] is repeated), the first one will create the lib directory where the download's libraries will be extracted to. However, the second will fail because the lib directory already exists. I just modified this to add exist_ok to the os.makedirs call so the second (and third, etc.) can add their libraries as well.